Skip to content

fix: residency-time-test: Check number of pc10 state entries#1354

Open
ekurdybx wants to merge 1 commit intothesofproject:mainfrom
ekurdybx:socwatch-test-fix
Open

fix: residency-time-test: Check number of pc10 state entries#1354
ekurdybx wants to merge 1 commit intothesofproject:mainfrom
ekurdybx:socwatch-test-fix

Conversation

@ekurdybx
Copy link
Contributor

Check how many times PC10 states was achieved and pass/fail test accordingly.

@ekurdybx ekurdybx force-pushed the socwatch-test-fix branch 2 times, most recently from f553843 to ef99326 Compare March 17, 2026 13:30
@ekurdybx ekurdybx marked this pull request as ready for review March 18, 2026 09:42
@ekurdybx ekurdybx requested review from a team, golowanow, lgirdwood and marc-hb as code owners March 18, 2026 09:42
# unload socwatch module
sudo bash "$SOCWATCH_PATH"/drivers/rmmod-socwatch

if [ "$pc10_achieved" = true ]; then
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When this is a local variable that you have full control on:

Suggested change
if [ "$pc10_achieved" = true ]; then
if "$pc10_achieved"; then

Check how many times PC10 states was achieved and pass/fail test
accordingly.

Signed-off-by: Emilia Kurdybelska <emiliax.kurdybelska@intel.com>
@ekurdybx ekurdybx force-pushed the socwatch-test-fix branch from ef99326 to bb588b2 Compare March 19, 2026 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants